Merge branch 'wip/nbenitezl/move-bookmark-placeholder-index-4' into 'master'

gtkplacessidebar.c: move bookmark at the placeholder index

See merge request GNOME/gtk!459

Merge branch 'patch-1' into 'master'

Zero-fill new GtkTextIter

See merge request GNOME/gtk!455

Merge branch 'wip/carlosg/imwayland-for-master' into 'master'

Use zwp_text_input_v3 on gtk4

Closes #1317 and #1525

See merge request GNOME/gtk!462

Fix XChangeProperty calls

They must be long...

Closes: #1556

Merge branch 'wip/carlosg/revealer-fix-interrupted-animations-master' into 'master'

revealer: Fully set the target state if unmapped during animation

See merge request GNOME/gtk!466

image: Avoid some unnecessary signal connections

Equivalent if fd9879e5ffba2c58b897155b39ff79584b3618e0 but for GtkImage.

image: Indentation fix

gl renderer: Add a clip stack

So we can check that the currently set clip is the first one and now
intersect with it. This first clip is always the entire viewport or the
entire render_area and we don't want to end up drawing things to a
texture because of it.

picture: Avoid some unnecessary signal connections

We don't need to ever invalidate the picture size if the paintable tells
us its size is static. Same for the contents.

image: Remove private struct from public header

CellAccessible: Fix return type of row|column_span

See:
 * c876c74eb73fe6daa279b42f9e0a9566ba1ced87
 * 63e0eb5de5d5a14afadebab31fe679beb6706cb2

Update Catalan translation

Update Catalan translation

Update Catalan translation

Update Catalan translation

Merge branch 'fix-typo-win32-compilation-speedup-macro-gtk4' into 'master'

Win32: Fix typo on compilation speedup macro define [Gtk4]

See merge request GNOME/gtk!473

Annotate values of PRIORITY constants

g-ir-scanner incorrectly evaluates macro definition that include
references to other macro definitions. Provide a correct value as an
annotation.

Differences in generated gir files:

```diff
@@ -19017 +19017 @@
-    <constant name="PRIORITY_REDRAW" value="20" c:type="GDK_PRIORITY_REDRAW">
+    <constant name="PRIORITY_REDRAW" value="120" c:type="GDK_PRIORITY_REDRAW">
@@ -74229,3 +74229,3 @@
     </constant>
-    <constant name="PRIORITY_RESIZE" value="10" c:type="GTK_PRIORITY_RESIZE">
+    <constant name="PRIORITY_RESIZE" value="110" c:type="GTK_PRIORITY_RESIZE">
       <doc xml:space="preserve">Use this priority for functionality related to size allocation.
@@ -106786,3 +106786,3 @@
     <constant name="TEXT_VIEW_PRIORITY_VALIDATE"
-              value="5"
+              value="125"
               c:type="GTK_TEXT_VIEW_PRIORITY_VALIDATE">
```

See !472

Update Catalan translation

Win32: Fix typo on compilation speedup macro define

There is a typo, the correct macro to define is WIN32_LEAN_AND_MEAN.
After this change <shellapi.h> must be included in order to use
ExtractIconExW().

mediastream: Fix possible memory leak

@error is (transfer full).  So the error passed should be freed if not used

revealer: Fully set the target state if unmapped during animation

If the revealer is told do animate and then unrealize itself, we do
(correctly) stop the animation, but used to do a shortcut where we
just set the target state as current.

Other things are dependent on the animation properly finishing though,
like the contained widget child visibility. This may lead to inconsistent
state where gtk_revealer_get_child_revealed() returns TRUE but the child
widget is unmapped, or vice-versa.

Fully finish the animation here, so the child state is coherent the next
time the revealer is mapped. We can also skip notifying on the property
since it will be handled by gtk_revealer_set_position().

x11: make the tool lookup dependent on the hw id as well

Tools on the same physical item have the same serial number, so the eraser
and the pen part of a single pen share that serial number. With the current
lookup code, we'll always return whichever tool comes first into proximity.

Change the code to use the hw id in addition to the serial number, this way we
can differ between two tools.

x11: don't add unknown tools to our list

Generic tools (Bamboo, built-in tablets) always have the same serial number
assigned by the wacom driver. This includes the touch tool when the wacom
driver handles the touch evdev node (common where users require the wacom
gestures to work).

When the first device is the touch device, a tool is created with that serial.
All future tools now return the touch tool on lookup since they all share the
same serial number. Worse, this happens *across* devices, so the pen
event node gets assigned the touch tool because they all have the same serial.

Since we don't actually care about the touch as a tool, let's skip any unknown
tool. This captures pads as well.

x11: get the tool type from the wacom driver properties

Any wacom device currently sets the tool type to UNKNOWN. The wacom driver has
a property that exports the tool type as one of stylus, eraser, cursor, pad or
touch. Only three of those are useful here but that's better than having all
of them as unknown.

build: wrap IM protocol building under have_wayland altogether

We poke variables that might not be defined if the wayland backend
is disabled.

imwayland: Respect maximum length of 4000 Bytes on strings being sent.

Hitting the limit will raise protocol errors.

imwayland: Plug leaks

The various strings (pending/current preedit, surrounding, and commit
buffer) are being leaked in the case of GtkIMContext destruction.

imwayland.c: fix formatting

imwayland: rearrange functions to remove prototypes

imwayland: Handle enter and leave events

Before this patch, imwayland would assume that text-input enter and leave events follow the general (wl_keyboard) focus, and was unable to handle the situation where they would not be provided at the same time.

imwayland: Collect return value from ::delete-surrounding signal

There's not much we can do about the signal not being handled, but
we should fetch the return value anyway.

imwayland: Don't reset serial while text-input is alive

The serial number is a persistent property of the text-input object.

imwayland: Fix clearing of preedit text in webkitgtk

Fixes webkitgtk misbehaviour as outlined in https://gitlab.gnome.org/GNOME/gtk/issues/1316#note_312942 , which was introduced in 49b17e6c.
The preedit will be cleared on exit only if it is already present.

imwayland: Fix clearing of preedit text

Fixes terminal emulator misbehaviour as outlined in https://gitlab.gnome.org/GNOME/gtk/issues/1316, which was introduced in 49b17e6c. The original commit cleared preedit text by setting it to an empty string, which still counted as existing preedit. The fix sets preedit string to null, which is correctly understood as not present.

modules: Check current context before retrieving surrounding

There may be situations where this might get called while the
currently focused context just went away (eg. after setting the
text widget unsensitive).

Closes: #1317

imwayland: clear preedit on focus out

imwayland: Add text-input-unstable-v3 support

The wayland input module now represents text-input-unstable-v3 support, while the old module supporting gtk-text-input was renamed to gtkwayland.

gstmedia: Implement error handling.

I wonder who forgot that.
Whoops.

mediastream: Allow multiple calls to gtk_media_stream_error()

Just ignore all further ones.

a11y: Fix function return type

Typo right there.

RGBA: tiny grammar improvements in to_string doc

RGBA: Consistently use “” around inline arg names

instead of being inconsistent and not using them later, which leaves a
bunch of single letters floating among real words, not the prettiest.

RGBA: Fix example to_string output for reality/CSS

* We don't output spaces anywhere in the code, unlike the doc suggested.
* CSS explicitly forbids whitespace between function names and lparens:
  https://stackoverflow.com/questions/13877198

gtkplacessidebar.c: move bookmark at the placeholder index

As that index is set in drag_motion_callback() and visually shown
on the widget as a drop target hint.

https://bugzilla.gnome.org/show_bug.cgi?id=787356
https://gitlab.gnome.org/GNOME/gtk/issues/904

ci: Update Docker image to Fedora 29 and meson to 0.49.0

Zero-fill new GtkTextIter

iter_init_common() is used on uninitialized GtkTextIter, and since neither it
nor its callers initiliaze its padding fields, they contain garbage.

This is a problem for Go - which checks that structs passed to C functions do
not contain pointers to Go-allocated memory - when the garbage happens to be
such a pointer.  Although Go zero-fills all GtkTextIter that it allocates, this
does not help when GTK functions such as insert_pixbuf_or_widget_segment called
for gtk_text_buffer_create_child_anchor copy garbage from their stack-allocated
GtkTextIter into a clean iter.  To work around this a GtkTextIter has to be
discraded after use in text buffer anchor inserting functions:
https://github.com/gotk3/gotk3/pull/307

Replace git.gnome.org by Gitlab URL in <GitRepository> in DOAP file

Replace Bugzilla by Gitlab URL in DOAP file

Remove gtk_widget_show_all() call

Extraneous chunk from 7601bca7587dd29bde3d1554e644a10ae6dafc18.

Cherry picked from gtk-3-24, which has a gtk_widget_show_all() function.

Merge branch 'master' into 'master'

A11y: Add support for AtkTableCell

See merge request GNOME/gtk!411

A11y: Add support for AtkTableCell

placesview: List only available protocols as available

We display a list of supported protocols in the server_addresses_popover.

However, this curated list contains protocols which may or may not be
available, depending on the respective gvfs backend being installed.

So, populate the list only with protocols which are available.

https://gitlab.gnome.org/GNOME/gtk/issues/1476

placesview: Set .error style if unsupported protocol

When the user types an address with a schema that is not supported,
the Connect button doesn't become sensitive, but there is no visible
feedback at all.

This feels unresponsive and leaves the user clueless.

While it doesn't help explain why the address doesn't work, this will
provide a hint that the input was acknowledged but doesn't work.

https://gitlab.gnome.org/GNOME/gtk/issues/1476

Merge branch 'fix-polygon-svg-recolor' into 'master'

icontheme: Recolor <polygon> elements in SVGs too

See merge request GNOME/gtk!443

Merge branch 'demo-combobox-typo-gtk4' into 'master'

demos/gtk-demo/combobox: fix typo

See merge request GNOME/gtk!444

Merge branch 'win32-honest-clipboard-gtk4' into 'master'

GDK W32: Be honest about supported clipboard formats (GTK4)

See merge request GNOME/gtk!399

Revert "Adwaita: GNOME 3.32"

This reverts all GNOME 3.32 styling from master

GDK W32: Always set gtk-font-name to the active UI font. Fixes #1484

This makes apps use "Segoe UI 9" by default instead of whatever matches "Sans 10".
It also cleans up the code and uses some new pango API while at it.

This was previously disabled in 9e686d1fb5cb20 because it led to a poor glyph coverage
on certain versions of Windows which don't default to "Segoe UI 9" (Chinese, Korean, ..)
because the font fallback list was missing in pango.

This is about to get fixed in https://gitlab.gnome.org/GNOME/pango/merge_requests/34
so enable it again when we detect a new enough pango version.

(See !436 for the original MR)

demos/gtk-demo/combobox: fix typo

Fix typo that prevented the P-S submenu from displaying correctly

Updated Slovenian translation

win32: Don't multiply the scroll event deltas by the Windows scroll lines setting. See #1408

GTK widgets expect the scroll deltas to be 1 or -1 and calculate a scroll value from that.
Multiplying the delta by the Windows scroll line setting (which defaults to 3) results
in a much larger delta and vastly different behaviour for running a GTK app on Windows
vs on Linux. For example text view and tree view scroll by 9 lines per scroll wheel tick
per default this way while on Linux it is around 3.

Remove the multiplication for now.

See !426 for the gtk3 MR

GDK W32: set default settings for fontconfig.

Enables hinting, antialiasing and set the subpixel orientation according to the
active clear type setting. This ensures that font rendering with the fontconfig backend
looks similar to the win32 backend, at least with the default system font.

See !437

icontheme: Recolor <polygon> elements in SVGs too

Otherwise, it's possible to have a symbolic icon where some of the
shapes keep the #bebebe chroma key color.

menushell: Listen to non-primary button clicks

Menus should also be deactivated on right-button clicks.

gl renderer: Track border width state separately

gl renderer: Ignore subsequent render target ops

gl renderer: Ignore viewport ops to the same viewport

gesturestylus: Use proper syntax when refering to signals

Adwaita: regenerate CSS

- for the previous patch

Merge branch 'context_menus' into 'master'

Adwaita GTK Theme: Add bigger shadow and border-radius to menus

See merge request GNOME/gtk!432

Adwaita: buttons & headerbar tweaks

- tone down the button z-depth
- flatter headerbars

Box, Grid: Improve various bits of documentation

Issue #1495 showed that the docs of GtkGrid retain outdated implications
that (as was once, but is no longer, the case) it is intended to replace
GtkBox, by discussing HfW and widget properties in a way that suggests
GtkBox can't handle them. But of course it does, and it's preferable for
simple single-row/column cases. Worse, we said GtkGrid “provides exactly
the same functionality” for the latter case, but the original point of
that Issues was that it doesn’t, at least for CSS positional selectors!

Box:
• Use an actually meaningful @Short_description.
• Remove unhelpful @See_also references to unrelated containers.
• Remove references to “rectangular area”: it might be another shape
  via CSS, or “rectangular” might falsely imply 2 dimensions of children.
• Mention Orientable:orientation.
• Emphasise usefulness of :[hv]align for allocating in the other axis.
• Don’t say that Grid “provides exactly the same functionality” for a
  single row or column, since (A) it is overkill for that case and (B)
  said Issue proved that it *doesn’t* for CSS child order, for example.

Grid:
• Don’t dwell on widget properties and height-for-width in a way that
  wrongly implies that Box can’t handle those (or Grid can better). In
  fact, just get rid of that bit altogether: Box handles them fine, and
  such wording was only needed years ago for migration from GTK+ 2 to 3.
• Point to GtkBox as being preferred for the simple row/column use case.

Merge branch 'bye-app-menu-gtk4' into 'master'

widget-factory: move app menu contents to primary menu (gtk4)

Closes #916

See merge request GNOME/gtk!365

application: Remove dead assignment

button: Remove some dead code

button: Remove some unnecessary includes

gl renderer: Fix remaining TODO about offset nodes

All of the nodes should now support offsets.

gl renderer: transform nodes: offset

gl renderer: Clip nodes: offset

gl renderer: Linear gradients: offset

gl renderer: Outset shadow nodes: offset

gl renderer: Care about offset when rendering border nodes

gl renderer: care about offset when rendering shadow nodes

emoji chooser: Actually force Emoji presentation

The previous commit was using the text presentation selector
instead of the Emoji one. Oops.

testuite Add translucent offscreen rendering test case

showrendernode: Fix GtkImage size

Make sure we show the paintable unscaled.

gl renderer: Remove blend shader

It's unused.

gl renderer: Reset opacity when rendering to a texture

The opacity will already be applied when rendering the final texture.

gl renderer: Add function to draw debug rectangle

gl renderer: Refactor render op builder

gl renderer: Add more node types to print_render_node_tree

debugging ++

window: Don't try to size-allocate unmapped popovers

The gtk_widget_size_allocate call won't do anything anyway.

gl renderer: Use ops_transform_bounds_modelview in more places

gl renderer: Add NodeSample

As a quick way of checking what a particular sample of nodes (e.g. all
offset node children) are made up of.

sizerequest: Directly query new request_mode

We are already inside the function that populates the size request
cache, so do it here instead of implicitly in get_request_mode.

gl renderer: Move geometry calculation further down

We don't need it above, so move it to where it belongs.

gl renderer: Apply offset with scale

gl renderer: Refactor add_offscreen_ops

Use a graphene_rect_t for the node bounds instead of 4 floats.
This makes it simpler to pass the size in without the offset applied.

gl renderer: Add offscreen ops without offset applied

When doing color matrix nodes. This fixes color matrix node with scale =
2.

gl renderer: Reset offset when rendering offscreen

We want to apply the offset to the rendered texture, not to the
offscreen-rendered content.